Skip to content

Fix problems caused by random project name generation#2619

Merged
labkey-tchad merged 1 commit intodevelopfrom
fb_randomProjectNameFix
Aug 13, 2025
Merged

Fix problems caused by random project name generation#2619
labkey-tchad merged 1 commit intodevelopfrom
fb_randomProjectNameFix

Conversation

@labkey-tchad
Copy link
Member

@labkey-tchad labkey-tchad commented Aug 11, 2025

Rationale

Changing TRICKY_CHARACTERS_FOR_PROJECT_NAMES caused a test to fail because it had the original value hard-coded.

java.lang.AssertionError: FAILED: test timechart-api.xml failed with status code: 404
{  "exception" : "No such folder or workbook: \/TimeChartTest Project☃~!@$&()_+{}-=[],.#äöüÅ\/Demo Study",  "success" : false}
  at org.junit.Assert.fail(Assert.java:89)
  at org.labkey.test.util.APITestHelper.sendRequestDirect(APITestHelper.java:186)
  at org.labkey.test.util.APITestHelper.runApiTests(APITestHelper.java:93)
  at org.labkey.test.util.APITestHelper.runApiTests(APITestHelper.java:74)
  at org.labkey.test.util.APITestHelper.runApiTests(APITestHelper.java:69)
  at org.labkey.test.tests.StudyBaseTest.runApiTests(StudyBaseTest.java:354)
  at org.labkey.test.tests.StudyBaseTest.testSteps(StudyBaseTest.java:361)

Also, the project name randomizer can generate illegal folder names. Updating random name generator to prevent these.

Related Pull Requests

Changes

  • Fix problems caused by random project name generation

@labkey-tchad labkey-tchad merged commit 6132161 into develop Aug 13, 2025
8 checks passed
@labkey-tchad labkey-tchad deleted the fb_randomProjectNameFix branch August 13, 2025 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@labkey-danield labkey-danield labkey-danield approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@labkey-tchad @labkey-danield